From 27965d5fdc56c91e6c252197d2497bf323b61e16 Mon Sep 17 00:00:00 2001 From: Benjamin Otte Date: Sun, 21 Nov 2021 01:51:35 +0100 Subject: [PATCH] builder-tool: Don't simplify enums too much Store the enum nick, not the enum value. That way the file remains human-readable. Updated reftests to new expected output. --- .../tools/simplify-data-3to4/box.expected | 136 +++++++++--------- .../simplify-data-3to4/checkbutton.expected | 2 +- .../tools/simplify-data-3to4/grid.expected | 8 +- .../gtkfilechooserdialog.expected | 4 +- .../simplify-data-3to4/office-runner.expected | 14 +- .../tools/simplify-data-3to4/toolbar.expected | 2 +- tools/gtk-builder-tool-simplify.c | 10 +- 7 files changed, 92 insertions(+), 84 deletions(-) diff --git a/testsuite/tools/simplify-data-3to4/box.expected b/testsuite/tools/simplify-data-3to4/box.expected index 54dde405b9..15b6f34c74 100644 --- a/testsuite/tools/simplify-data-3to4/box.expected +++ b/testsuite/tools/simplify-data-3to4/box.expected @@ -7,15 +7,15 @@ 1 - 1 + vertical - 3 + center 1 0 - 3 + center _ 1 1 @@ -30,7 +30,7 @@ - 3 + center 1 E 1 @@ -50,12 +50,12 @@ 0 - 3 + center 1 0 - 3 + center _ 1 1 @@ -70,7 +70,7 @@ - 3 + center 1 E 1 @@ -91,13 +91,13 @@ - 3 - 1 + center + vertical 0 1 - 3 + center _ 1 1 @@ -112,7 +112,7 @@ - 3 + center 1 E 1 @@ -133,10 +133,10 @@ 0 1 - 1 + vertical - 3 + center _I 1 1 @@ -151,7 +151,7 @@ - 3 + center 1 EI 1 @@ -171,7 +171,7 @@ _S 1 1 - 3 + center @@ -179,7 +179,7 @@ FS 1 1 - 1 + start @@ -188,7 +188,7 @@ ES 1 1 - 3 + center @@ -197,7 +197,7 @@ FES 1 1 - 1 + start @@ -205,7 +205,7 @@ _N 1 1 - 3 + center @@ -213,7 +213,7 @@ FN 1 1 - 2 + end @@ -222,7 +222,7 @@ EN 1 1 - 3 + center @@ -231,7 +231,7 @@ FEN 1 1 - 2 + end @@ -239,7 +239,7 @@ _C 1 1 - 3 + center @@ -247,7 +247,7 @@ FC 1 1 - 3 + center @@ -256,7 +256,7 @@ EC 1 1 - 3 + center @@ -265,7 +265,7 @@ FEC 1 1 - 3 + center @@ -273,7 +273,7 @@ _B 1 1 - 3 + center @@ -281,7 +281,7 @@ FB 1 1 - 4 + baseline @@ -290,7 +290,7 @@ EB 1 1 - 3 + center @@ -299,7 +299,7 @@ FEB 1 1 - 4 + baseline @@ -308,10 +308,10 @@ 0 1 - 1 + vertical - 3 + center _I 1 1 @@ -328,7 +328,7 @@ - 3 + center EI 1 1 @@ -348,7 +348,7 @@ _S 1 1 - 3 + center 0 @@ -357,7 +357,7 @@ FS 1 1 - 1 + start 0 @@ -366,7 +366,7 @@ ES 1 1 - 3 + center 1 @@ -375,7 +375,7 @@ FES 1 1 - 1 + start 1 @@ -384,7 +384,7 @@ _N 1 1 - 3 + center 0 @@ -393,7 +393,7 @@ FN 1 1 - 2 + end 0 @@ -402,7 +402,7 @@ EN 1 1 - 3 + center 1 @@ -411,7 +411,7 @@ FEN 1 1 - 2 + end 1 @@ -420,7 +420,7 @@ _C 1 1 - 3 + center 0 @@ -429,7 +429,7 @@ FC 1 1 - 3 + center 0 @@ -438,7 +438,7 @@ EC 1 1 - 3 + center 1 @@ -447,7 +447,7 @@ FEC 1 1 - 3 + center 1 @@ -456,7 +456,7 @@ _B 1 1 - 3 + center 0 @@ -465,7 +465,7 @@ FB 1 1 - 4 + baseline 0 @@ -474,7 +474,7 @@ EB 1 1 - 3 + center 1 @@ -483,7 +483,7 @@ FEB 1 1 - 4 + baseline 1 @@ -493,10 +493,10 @@ 0 1 - 1 + vertical - 3 + center _I 1 1 @@ -513,7 +513,7 @@ - 3 + center EI 1 1 @@ -533,7 +533,7 @@ _S 1 1 - 3 + center 1 @@ -542,7 +542,7 @@ FS 1 1 - 1 + start 1 @@ -551,7 +551,7 @@ ES 1 1 - 3 + center 1 @@ -560,7 +560,7 @@ FES 1 1 - 1 + start 1 @@ -569,7 +569,7 @@ _N 1 1 - 3 + center 1 @@ -578,7 +578,7 @@ FN 1 1 - 2 + end 1 @@ -587,7 +587,7 @@ EN 1 1 - 3 + center 1 @@ -596,7 +596,7 @@ FEN 1 1 - 2 + end 1 @@ -605,7 +605,7 @@ _C 1 1 - 3 + center 1 @@ -614,7 +614,7 @@ FC 1 1 - 3 + center 1 @@ -623,7 +623,7 @@ EC 1 1 - 3 + center 1 @@ -632,7 +632,7 @@ FEC 1 1 - 3 + center 1 @@ -641,7 +641,7 @@ _B 1 1 - 3 + center 1 @@ -650,7 +650,7 @@ FB 1 1 - 4 + baseline 1 @@ -659,7 +659,7 @@ EB 1 1 - 3 + center 1 @@ -668,7 +668,7 @@ FEB 1 1 - 4 + baseline 1 diff --git a/testsuite/tools/simplify-data-3to4/checkbutton.expected b/testsuite/tools/simplify-data-3to4/checkbutton.expected index 24ef3bcd37..c5f8ed6361 100644 --- a/testsuite/tools/simplify-data-3to4/checkbutton.expected +++ b/testsuite/tools/simplify-data-3to4/checkbutton.expected @@ -4,7 +4,7 @@ - 1 + vertical 1 diff --git a/testsuite/tools/simplify-data-3to4/grid.expected b/testsuite/tools/simplify-data-3to4/grid.expected index 60f2972846..2d040d0f19 100644 --- a/testsuite/tools/simplify-data-3to4/grid.expected +++ b/testsuite/tools/simplify-data-3to4/grid.expected @@ -100,8 +100,8 @@ 1 1 - 3 - 3 + center + center 1 1 @@ -112,8 +112,8 @@ 1 1 - 3 - 3 + center + center 2 1 diff --git a/testsuite/tools/simplify-data-3to4/gtkfilechooserdialog.expected b/testsuite/tools/simplify-data-3to4/gtkfilechooserdialog.expected index 9275810c54..b96ec784d8 100644 --- a/testsuite/tools/simplify-data-3to4/gtkfilechooserdialog.expected +++ b/testsuite/tools/simplify-data-3to4/gtkfilechooserdialog.expected @@ -9,7 +9,7 @@ 0 - 1 + vertical 0 @@ -31,6 +31,6 @@ - 2 + vertical diff --git a/testsuite/tools/simplify-data-3to4/office-runner.expected b/testsuite/tools/simplify-data-3to4/office-runner.expected index b166475564..fa8453830d 100644 --- a/testsuite/tools/simplify-data-3to4/office-runner.expected +++ b/testsuite/tools/simplify-data-3to4/office-runner.expected @@ -11,7 +11,7 @@ 12 12 12 - 1 + vertical 12 @@ -34,7 +34,7 @@ - 1 + start 1 0 • Click the "Run!" button, and then dash to your desk or meeting. @@ -129,11 +129,11 @@ - 3 - 1 + center + vertical - 1 + start 0 Silver Trophy! 1 @@ -148,7 +148,7 @@ - 1 + start 0 You managed to finish the route with the 2nd best time ever, <b>48.15 seconds!</b> 1 @@ -157,7 +157,7 @@ - 1 + start 0 Only <b>3.75 seconds</b> separate you from the gold trophy! 1 diff --git a/testsuite/tools/simplify-data-3to4/toolbar.expected b/testsuite/tools/simplify-data-3to4/toolbar.expected index d1bafefd5c..6b5db81054 100644 --- a/testsuite/tools/simplify-data-3to4/toolbar.expected +++ b/testsuite/tools/simplify-data-3to4/toolbar.expected @@ -12,7 +12,7 @@ - 1 + vertical diff --git a/tools/gtk-builder-tool-simplify.c b/tools/gtk-builder-tool-simplify.c index 0f13bf7a6d..dfa4eb7097 100644 --- a/tools/gtk-builder-tool-simplify.c +++ b/tools/gtk-builder-tool-simplify.c @@ -621,7 +621,15 @@ canonical_enum_value (MyParserData *data, GValue value = G_VALUE_INIT; if (gtk_builder_value_from_string_type (data->builder, type, string, &value, NULL)) - return g_strdup_printf ("%d", g_value_get_enum (&value)); + { + GEnumClass *eclass = g_type_class_ref (type); + GEnumValue *evalue = g_enum_get_value (eclass, g_value_get_enum (&value)); + + if (evalue) + return g_strdup (evalue->value_nick); + else + return g_strdup_printf ("%d", g_value_get_enum (&value)); + } return NULL; } -- 2.30.2